Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Create curve pkg and organize remaining files #394

Merged
merged 9 commits into from
Oct 6, 2023
Merged

Create curve pkg and organize remaining files #394

merged 9 commits into from
Oct 6, 2023

Conversation

cicr99
Copy link
Contributor

@cicr99 cicr99 commented Oct 5, 2023 •
edited
Loading

  • Created new packages for:
    • curve
    • typed
    • merkle
  • providers_test was removed as there was already pretty much the same test in the rpc package
  • Example for curve was removed as it was no longer needed
  • Fixed broken example for deploy account (because of the curve package and a previous change in BigIntToArr method

Closes #393

@cicr99 cicr99 self-assigned this Oct 5, 2023
@cicr99 cicr99 requested a review from rianhughes October 5, 2023 10:03
rianhughes
rianhughes reviewed Oct 6, 2023
View reviewed changes
curve/utils_test.go Outdated Show resolved Hide resolved
rianhughes
rianhughes reviewed Oct 6, 2023
View reviewed changes
account/hash.go Outdated Show resolved Hide resolved
@rianhughes rianhughes merged commit 61e7f8f into main Oct 6, 2023
2 of 3 checks passed
@rianhughes rianhughes deleted the restructuring-curve_pkg branch October 6, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rianhughes rianhughes rianhughes approved these changes

Assignees

@cicr99 cicr99

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

move all EC logic into a pkg
2 participants
@cicr99 @rianhughes